New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Requested JsHelper buffer #156
Conversation
Sorry, only the last commit is to be taken into account : PathMotion@4d9afe8 I don't know how to put only this one here :/ |
@@ -223,7 +223,7 @@ class DataSource extends Object { | |||
* @return array Array of sources available in this datasource. | |||
* @access public | |||
*/ | |||
function listSources($data = null) { | |||
function cachedListSources($data = null) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Won't this break a pile of backwards compatibility?
Seems like there are a bunch of other changes that are not part of the original pull request in here as well. Putting things on separate branches makes the pull requests work better. |
hello CakePHP team.
While switching to the JsHelper, which is great to optimize the javascript code.
I had a problem with string bufferised from a requested action : the two requests (main one [http] and second one [requested] ) have each their buffer and it's not shared.
So some Js strings from requested ones are not in the main output.
I guess it's a known beahvior (each request have their own helpers). But if you consider to change this (at least for the Js Helper buffer), here is a patch to do this :
At the end, the "main" JsHelper collected all the buffers in one and can output it.